Skip to content

Conversation

Brijeshthummar02
Copy link
Contributor

issue #28

i have tested it locally

Screenshot 2025-02-12 184303

@Brijeshthummar02
Copy link
Contributor Author

@dearrudam update on this one as well.

Copy link
Contributor

@dearrudam dearrudam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Brijeshthummar02 thanks for your contribution👏 ! Could you improve the other ones as well: java-11.yml and java-17.yml?

@Brijeshthummar02
Copy link
Contributor Author

@dearrudam Sure! but what i suggest for java-11.yml and java-17.yml there needs to be 2 more new PR as it will help to maintain the structure and if anything goes off we can make changes specifically for that PR.

@Brijeshthummar02
Copy link
Contributor Author

@dearrudam this PR is perfect right we don't need changes in this one right only for java 11-17.

@dearrudam dearrudam dismissed their stale review February 18, 2025 12:21

The requested changes will be done in additional separated PRs.

@dearrudam dearrudam merged commit 1c7eb59 into soujava:main Feb 18, 2025
4 checks passed
@Brijeshthummar02 Brijeshthummar02 deleted the java-21 branch February 18, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants